Skip to content

test(mobile): add unit tests for mobile QR scanner screen and hook - #928

Merged
johdanike merged 1 commit into
SwiftChainn:mainfrom
TechBroAfrica:test/mobile-qr-scanner-tests
Oct 23, 2026
Merged

johdanike merged 1 commit into
SwiftChainn:mainfrom
TechBroAfrica:test/mobile-qr-scanner-tests

Conversation

@TechBroAfrica

@TechBroAfrica TechBroAfrica commented Oct 18, 2026 •

Copy link
Copy Markdown

Closes #829

test(mobile): add QR scanner screen and hook coverage

PR Description
Summary
This PR adds the missing mobile QR scanner screen and hook coverage for the SwiftChain frontend.

What was implemented
Added the mobile QR scanner screen UI with:
permission loading state
live camera feed
retry flow for camera access errors
successful scan result overlay
manual tracking-code fallback
offline watermark state
Added the mobile QR scanner hook to handle:
camera start/stop logic
permission errors
QR verification request flow
scan result storage
Added the API service layer for QR verification
Added the supporting TypeScript model for scanned delivery data
Test coverage
Added focused unit tests covering:

permission loading state
camera error state with retry button
successful QR scan result display
manual entry flow
offline watermark rendering
hook-level scan and permission handling
Verification
Ran:
cd "/home/semicolon/Desktop/TechBroAfrica Drip/SwiftChain_Frontend" && npm test -- --runInBand components/mobile/tests/QrScannerScreen.test.tsx

Result:

1 test suite passed
9 tests passed
0 failed
Closes
Closes #[issue_id]

If you want, I can also tailor this into a more polished GitHub PR format with a checklist and a shorter “What Changed / Why / Validation” structure.

@drips-wave

drips-wave Bot commented Oct 18, 2026

Copy link
Copy Markdown

@TechBroAfrica Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@johdanike

Copy link
Copy Markdown
Contributor

Please always follow the laid down guidelines as directed in the issue description.
Issues are closed using the upper case letter form for the 'c' and not the lower as used in your pr. I'd fix that but note for next time.

@johdanike

Copy link
Copy Markdown
Contributor

Again, thank you for contributing to Swiftchainn

@johdanike johdanike left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nicely done

@johdanike
johdanike merged commit d5623c5 into SwiftChainn:main Oct 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Test: Mobile QR Scanner Screen and Hook Tests

2 participants